Skip to content

Add HeaderConfiguration XML decoding tests#33

Merged
powerje merged 1 commit intover/4.0.0from
header-configuration-decoding-tests-v4
Apr 7, 2026
Merged

Add HeaderConfiguration XML decoding tests#33
powerje merged 1 commit intover/4.0.0from
header-configuration-decoding-tests-v4

Conversation

@VenkataKrishnaGarapati
Copy link
Copy Markdown

@VenkataKrishnaGarapati VenkataKrishnaGarapati commented Apr 1, 2026

Summary

  • Added decoding tests for Atom feed and Configuration payload
  • Handled empty XML elements (Choices, ReadOnlyCriteria)
  • All tests passing

Files Added

File Purpose
Tests/KumoTests/Mocks/XML/HeaderConfiguration.swift Decodable model structs (Configuration, Control, Criterion, Column, etc.) plus Atom feed wrappers
Tests/KumoTests/Fixtures/XML/HeaderConfigurationDecodingTests.swift 4 XCTest cases: single config, empty ReadOnlyCriteria, empty Choices, full Atom feed

Key Adjustments from Genspec Output

  • FieldFieldModel rename to avoid Swift property/type name collision
  • Custom HeaderConfigurationFeed.init(from:) — unkeyed container loop to collect all <entry> siblings (KumoCoding keyed container only finds the first match)

Replaces #32 (was incorrectly targeting main)

@powerje powerje changed the title Add HeaderConfiguration XML decoding tests using sanitized eSOne structure Add HeaderConfiguration XML decoding tests Apr 3, 2026
@VenkataKrishnaGarapati VenkataKrishnaGarapati force-pushed the header-configuration-decoding-tests-v4 branch from 0fb2ebb to 662ec07 Compare April 7, 2026 21:13
@powerje powerje merged commit b26a5c4 into ver/4.0.0 Apr 7, 2026
4 checks passed
@powerje powerje deleted the header-configuration-decoding-tests-v4 branch April 7, 2026 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants